-
Notifications
You must be signed in to change notification settings - Fork 23.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FieldAttribute
s as descriptors
#73908
Conversation
This comment has been minimized.
This comment has been minimized.
|
I was thinking the same, the only reason I have it this way for now is that I didn't know how to name the noninharitable :-)
+1 |
This comment has been minimized.
This comment has been minimized.
555d880
to
8420e71
Compare
2292474
to
c54cb02
Compare
23586a9
to
ab7c59b
Compare
8f03f4d
to
6e0c6c3
Compare
6e0c6c3
to
9591860
Compare
become_method = FieldAttribute(isa='string', default=context.cliargs_deferred_get('become_method')) | ||
become_user = FieldAttribute(isa='string', default=context.cliargs_deferred_get('become_user')) | ||
become_flags = FieldAttribute(isa='string', default=context.cliargs_deferred_get('become_flags')) | ||
become_exe = FieldAttribute(isa='string', default=context.cliargs_deferred_get('become_exe')) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a blocker but 'cliargs_deffered_get' paths could be simplified by just setting it as part of inheritance in the run, default hte 'top object' (currently play, but could be 'playbook') to the cli/config/env var values.
ci_complete
ci_complete
65c90bb
to
25fe945
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
SUMMARY
FieldAttribute
s as descriptorsISSUE TYPE
COMPONENT NAME
many